Added Scroll Up button at the bottom of the homepage#35
Open
nageshkharat wants to merge 1 commit intokeploy:mainfrom
Open
Added Scroll Up button at the bottom of the homepage#35nageshkharat wants to merge 1 commit intokeploy:mainfrom
nageshkharat wants to merge 1 commit intokeploy:mainfrom
Conversation
Signed-off-by: nageshkharat <kharatnagesh049@gmail.com>
There was a problem hiding this comment.
Pull Request Overview
This PR adds a “Scroll Up” button to the bottom-right corner of the homepage to improve user navigation.
- Renames the existing scroll-down container class to home-content__scroll_down.
- Inserts a new scroll up button in index.html and adds corresponding CSS styling in main.css.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| index.html | Updated scroll-down container class and added scroll up button markup. |
| css/main.css | Renamed CSS selectors for scroll-down and added new styles for scroll up. |
| transition: all 0.15s ease-in-out; | ||
| } | ||
|
|
||
| .home-content__scroll_up { |
There was a problem hiding this comment.
The .home-content__scroll_up rule does not specify a bottom position, which may cause inconsistent placement of the button. Consider adding a bottom property similar to the scroll-down button to ensure it appears at the bottom-right as intended.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Author
|
@Achanandhi-M Hey! I’ve submitted a PR for the Scroll Up button enhancement. Please take a look when you get time. |
|
Hey @nageshkharat, Thank you for raising the pr! Our team will look into it soon. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Template
Description
Added a Scroll Up button to the bottom-right corner of the website. This enhances user navigation by allowing them to smoothly scroll back to the top after reaching the bottom of the page. The button is styled similarly to the existing Scroll Down button and appears only at the end of the page.
Fixes # Enhancement idea for better UX (no open issue linked)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Additional Context (Please include any Screenshots/gifs if relevant)
Here is a quick screenshot showing the Scroll Up button placement and styling at the page bottom:
Checklist: